Skip to content

feat: extract text from PreviewFormBanner and mutation files into sep…#9266

Open
brendancjz wants to merge 3 commits intoopengovsg:developfrom
brendancjz:feat/i18n-extract-text-form-banner-and-mutations
Open

feat: extract text from PreviewFormBanner and mutation files into sep…#9266
brendancjz wants to merge 3 commits intoopengovsg:developfrom
brendancjz:feat/i18n-extract-text-form-banner-and-mutations

Conversation

@brendancjz
Copy link
Copy Markdown
Contributor

Problem

Closes #8454

Solution

  • Extract text into i18n keys

Breaking Changes

  • Yes - this PR contains breaking changes
    • Details ...
  • No - this PR is backwards compatible

Features:

  • Details ...

Improvements:

  • Details ...

Bug Fixes:

  • Details ...

Before & After Screenshots

BEFORE:

AFTER:

Tests

Deploy Notes

New environment variables:

  • env var : env var details

New scripts:

  • script : script details

New dependencies:

  • dependency : dependency details

New dev dependencies:

  • dependency : dependency details

@brendancjz brendancjz requested a review from a team as a code owner April 1, 2026 04:57
@brendancjz
Copy link
Copy Markdown
Contributor Author

brendancjz commented Apr 1, 2026

Hey @LoneRifle for your review. Tried running pnpm lint but has no feedback logs and was still running after 30mins.
image

Running pnpm lint-staged did not show any linting errors so it should be good. Let me know if the CI pipeline fails.
image

Copy link
Copy Markdown
Contributor

@LoneRifle LoneRifle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct these first - i'll look into the PR again once done

Comment thread apps/frontend/src/features/admin-form/assistance/mutations.ts Outdated
Comment thread apps/frontend/src/features/admin-form/common/mutations.ts Outdated
Comment thread apps/frontend/src/features/admin-form/common/mutations.ts Outdated
@brendancjz
Copy link
Copy Markdown
Contributor Author

correct these first - i'll look into the PR again once done

Hey @LoneRifle, made the changes. Let me know if you have further comments thanks!

@LoneRifle
Copy link
Copy Markdown
Contributor

The Forms team is backed up and they haven't gotten round to merging your other PRs to mainline. Deliberately introducing some backpressure here so that they don't get overwhelmed. Hang tight for now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(i18n): extract text from PreviewFormBanner component & mutation in common and assistance of admin-from

2 participants